Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix backwards cmpxchg_weak_failure_rate check #2105

Merged
merged 1 commit into from
May 23, 2022

Conversation

thomcc
Copy link
Member

@thomcc thomcc commented May 7, 2022

Uh, I guess I got this backwards a long time ago 😓.

src/data_race.rs Outdated Show resolved Hide resolved
@cbeuw
Copy link
Contributor

cbeuw commented May 8, 2022

Ah this is why crossbeam had to set -Zmiri-compare-exchange-weak-failure-rate=1.0 to fix Miri hanging on tests crossbeam-rs/crossbeam#578 (comment), which made no sense. @taiki-e would need to change that back to -Zmiri-compare-exchange-weak-failure-rate=0.0 (I pinned down why this was needed in the first place, will post it in a separate crossbeam issue)

src/data_race.rs Outdated Show resolved Hide resolved
@RalfJung RalfJung added the S-waiting-on-author Status: Waiting for the PR author to address review comments label May 17, 2022
@thomcc thomcc force-pushed the flip-failure-rate branch from 5bac6ff to 7bbaada Compare May 23, 2022 03:22
@thomcc thomcc force-pushed the flip-failure-rate branch from 7bbaada to fdfbd7a Compare May 23, 2022 03:24
@RalfJung
Copy link
Member

@bors r+

@bors
Copy link
Contributor

bors commented May 23, 2022

📌 Commit fdfbd7a has been approved by RalfJung

@bors
Copy link
Contributor

bors commented May 23, 2022

⌛ Testing commit fdfbd7a with merge 72e11d3...

@RalfJung
Copy link
Member

This reminds me, is there some test we could add for this?

@bors
Copy link
Contributor

bors commented May 23, 2022

☀️ Test successful - checks-actions
Approved by: RalfJung
Pushing 72e11d3 to master...

@RalfJung
Copy link
Member

Added a test in #2148

bors added a commit that referenced this pull request May 23, 2022
…t, r=RalfJung

test that compare-exchange-weak-failure-rate=0.0 means what it says

Adds a test for #2105
@thomcc thomcc deleted the flip-failure-rate branch May 23, 2022 14:36
bors bot added a commit to crossbeam-rs/crossbeam that referenced this pull request May 31, 2022
843: Fix miri tests r=taiki-e a=alygin

There were several changes in Miri recently, including [breaking ones](rust-lang/miri#2105) and affecting its speed. They caused CI builds failures.

This fix makes the most expensive Miri tests less intensive, so they have decent execution time again.


Co-authored-by: Andrew Lygin <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-author Status: Waiting for the PR author to address review comments
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants